Skip to content

Conversation

dulmandakh
Copy link
Contributor

@dulmandakh dulmandakh commented Jun 29, 2024

This PR migrates legacy packaging to Poetry, making dependency management easy.

Depends on:

@erikwrede
Copy link
Member

I'd personally prefer switching to poetry or pdm right away when we make the switch to pyproject.toml What do you think? @dulmandakh

@dulmandakh
Copy link
Contributor Author

I use poetry for my projects. I can help with migration too.

I'll change the PR to use poetry then 😉

@dulmandakh dulmandakh changed the title migrate to pyproject.toml migrate to poetry Jul 5, 2024
@dulmandakh
Copy link
Contributor Author

@erikwrede please review 😉

@@ -46,7 +46,6 @@
from .utils.resolve_only_args import resolve_only_args

__all__ = [
"__version__",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's important to keep __version__, as certain extension libraries use it do distignuish between different grapehene releases. I'm fine with deprecation, as python 3.8's importlib.metadata.version('graphene') is a viable alternative. I'd suggest using importlib to fill __version__ and deprecating it, recommending direct use of importlib instead. What do you think? 😊

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll look into it and make changes accordingly

Comment on lines -48 to -53
def get_docs_version(version=None):
version = get_complete_version(version)
if version[3] != "final":
return "dev"
else:
return "%d.%d" % version[:2]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Its also important we keep these in the same format to ensure our docs pipeline doesn't break. Unfortunately, I cannot change much about the way the docs are deployed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants